Skip to content

Conversation

Kguswo
Copy link
Contributor

@Kguswo Kguswo commented Jul 18, 2025

Replace switch statement with LogLevel.log() method call in DeferredLog.logTo().

Changes:

  • Use level.log(log, message, throwable) instead of switch statement

Motivation:

  • Reduces code duplication
  • Improves maintainability
  • Cleaner, more concise implementation

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 18, 2025
@Kguswo Kguswo force-pushed the improve-deferred-log-method branch from edac69d to da298cd Compare July 21, 2025 08:52
@philwebb philwebb self-assigned this Jul 21, 2025
@philwebb philwebb added type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged labels Jul 21, 2025
@philwebb philwebb added this to the 3.4.x milestone Jul 21, 2025
@snicoll snicoll added the for: merge-with-amendments Needs some changes when we merge label Jul 25, 2025
@snicoll snicoll self-assigned this Aug 13, 2025
@snicoll snicoll modified the milestones: 3.4.x, 3.4.9 Aug 13, 2025
snicoll pushed a commit that referenced this pull request Aug 13, 2025
@snicoll snicoll closed this in 7af646e Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
for: merge-with-amendments Needs some changes when we merge type: task A general task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants